Summary

Describe the change.

What Changed

Key files and behaviors.

Validation

How you tested.

Risks

Anything to watch.

  Open in ADE  ·  ade/cursor-pty-brain-flaps-cdd125fe branch  ·  PR #565

Greptile Summary

This PR changes how Cursor CLI sessions receive their initial prompt: instead of embedding the prompt (and ADE guidance) as a trailing command-line argument to cursor-agent, the text is now submitted through PTY input after the interactive CLI is ready, with a 750 ms readiness delay. Empty Cursor launches no longer auto-submit ADE guidance as the first turn.

• cliLaunch.ts: The Cursor branch of buildTrackedCliLaunchCommand drops the prompt from args and conditionally sets initialInput/initialInputDelayMs: 750 only when an initialPrompt is present; empty launches produce a clean idle session.
• Tests (cliLaunch.test.ts, adeRpcServer.test.ts, syncRemoteCommandService.test.ts, AgentChatPane.test.tsx): All relevant assertions flipped from checking prompt membership in args to checking initialInput/initialInputDelayMs, and a new test case covers the empty-launch idle path.
• Docs: README.md, pty-and-processes.md, and ui-surfaces.md updated to describe the new launch strategy and the selective ADE guidance injection for Cursor.

Confidence Score: 5/5

Safe to merge; the fresh-launch path for Cursor is cleanly implemented and thoroughly tested across all relevant test suites.

All changed test assertions are consistent with the implementation. The one area worth a follow-up is whether buildTrackedCliResumeCommand for Cursor (unchanged in this PR) also needs to stop appending the prompt to args, given the docs now describe that path as prompt-free — but that is a documentation/code alignment question for the resume path, not a defect in the fresh-launch change being shipped here.

The resume branch for Cursor in buildTrackedCliResumeCommand (cliLaunch.ts lines 803–819) may be worth a follow-up to confirm it aligns with the updated docs.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant ADE as ADE Runtime
    participant Builder as buildTrackedCliLaunchCommand
    participant PTY as PTY Service
    participant Cursor as cursor-agent process

    ADE->>Builder: "provider="cursor", initialPrompt?"

    alt initialPrompt present
        Builder-->>ADE: "{ args:[--mode,--model,...], initialInput:"ADE guidance\nUser prompt", initialInputDelayMs:750 }"
        ADE->>PTY: "create(command="cursor-agent", args, initialInput, initialInputDelayMs=750)"
        PTY->>Cursor: spawn cursor-agent [--mode] [--model]
        Cursor-->>PTY: interactive prompt ready
        PTY->>Cursor: write initialInput after 750ms delay
    else no initialPrompt (empty launch)
        Builder-->>ADE: "{ args:[--model,...] } — no initialInput"
        ADE->>PTY: "create(command="cursor-agent", args)"
        PTY->>Cursor: spawn cursor-agent [--model] — idle, awaits user
    end

_{Reviews (2): Last reviewed commit: "Fix Cursor Work draft launch tests" | Re-trigger Greptile}